Skip to content

Conversation

s-urbaniak
Copy link
Collaborator

Summary

This removes presumably unnecessary manifest artifacts that are supposed to be generated during releases only.

Proof of Work

Passing e2e tests.

Checklist

  • Have you linked a jira ticket and/or is the ticket in the title?
  • Have you checked whether your jira ticket required DOCSP changes?
  • Have you checked for release_note changes?
  • Have you signed our CLA?

Reminder (Please remove this when merging)

  • Please try to Approve or Reject Changes the PR, keep PRs in review as short as possible
  • Remember the following Communication Standards - use comment prefixes for clarity:
    • blocking: Must be addressed before approval.
    • follow-up: Can be addressed in a later PR or ticket.
    • q: Clarifying question.
    • nit: Non-blocking suggestions.
    • note: Side-note, non-actionable. Example: Praise
    • --> no prefix is considered a question

@s-urbaniak s-urbaniak requested a review from a team as a code owner October 7, 2025 06:31
@s-urbaniak s-urbaniak changed the title CLOUDP-349340: transient build for generated manifests WIP CLOUDP-349340: transient build for generated manifests Oct 7, 2025
@s-urbaniak s-urbaniak added the hold Do not merge for now label Oct 7, 2025
@s-urbaniak s-urbaniak marked this pull request as draft October 7, 2025 06:36
@s-urbaniak s-urbaniak removed the hold Do not merge for now label Oct 8, 2025
@s-urbaniak s-urbaniak changed the title WIP CLOUDP-349340: transient build for generated manifests CLOUDP-349340: transient build for generated manifests Oct 8, 2025
@s-urbaniak s-urbaniak marked this pull request as ready for review October 8, 2025 16:36
@s-urbaniak s-urbaniak added cloud-tests Run expensive Cloud Tests: Integration & E2E test/e2e2/* labels Oct 8, 2025
Copy link
Collaborator

@josvazg josvazg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I don't care about the codecov complain, let's go!

@roothorp roothorp merged commit ff1ee9d into main Oct 13, 2025
219 of 222 checks passed
@roothorp roothorp deleted the transient-build branch October 13, 2025 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cloud-tests Run expensive Cloud Tests: Integration & E2E test/e2e2/*

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants